Skip to content

Jedis instrumentation. - #1757

Merged
arcuri82 merged 9 commits into
masterfrom
feature/jedis-integration
Sep 17, 2026
Merged

arcuri82 merged 9 commits into
masterfrom
feature/jedis-integration

Conversation

@aszyrej

@aszyrej aszyrej commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Adding Jedis support for new case studies to be added.

  • New FT.* commands added to support new possible case study.
  • New JSON.* commands and others added to support possible candidate case studies.
  • Refactor on where command arguments are being parsed.
  • Added ConnectionClassReplacement.
  • New line in ReplacementList.
  • Unit tests.

@aszyrej
aszyrej requested a review from jgaleotti September 14, 2026 14:48
@aszyrej
aszyrej requested a review from jgaleotti September 16, 2026 02:22
@jgaleotti
jgaleotti requested a review from arcuri82 September 16, 2026 13:45
@@ -1,5 +1,7 @@
package org.evomaster.client.java.controller.redis;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aszyrej @jgaleotti shouldn't we have at least 1 E2E test to verify these new functionalities?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aszyrej @jgaleotti or is Redis support not there yet for having first E2Es on it?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arcuri82 We are creating separate PRs for adding the e2e tests (to make the PRs shorter).
Should e2e tests be included in the same PR where an experimental functionality is being added?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jgaleotti it is fine, separated PR is OK for now. but for future, make sense to have them together. can't be sure an implementation is correct if no E2E yet :)

@arcuri82
arcuri82 merged commit bd6724e into master Sep 17, 2026
31 checks passed
@arcuri82
arcuri82 deleted the feature/jedis-integration branch September 17, 2026 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants